Skip to content

add testing#58

Merged
matthewangel merged 1 commit into
masterfrom
switchback
May 20, 2026
Merged

add testing#58
matthewangel merged 1 commit into
masterfrom
switchback

Conversation

@matthewangel

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings May 20, 2026 18:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the nf-test expectations and adds a Nextflow stub: implementation for the find_switchback_reads module so the workflow can run under -stub-run while still producing the expected switchbacks outputs.

Changes:

  • Added a stub: block to modules/find_switchback_reads.nf that creates placeholder .switchbacks.bam and .switchbacks.summary.txt outputs.
  • Updated tests/main.nf.test.snap to reflect the additional tasks and the new --fgbio_switchbacks argument appearing in the aggregated arguments output.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
tests/main.nf.test.snap Updates snapshots for task counts and aggregated arguments to include the switchbacks summary input.
modules/find_switchback_reads.nf Adds a stub: script to generate placeholder switchbacks outputs during -stub-run testing.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@matthewangel
matthewangel merged commit d07ce8a into master May 20, 2026
6 checks passed
@matthewangel
matthewangel deleted the switchback branch May 20, 2026 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants